Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 15, 2026

Merged main branch into PR branch to resolve divergence. Two files had conflicts requiring manual resolution.

Conflict Resolutions

src/backends/traefik/templateParser.ts

  • Merged enhanced JSDoc documentation from PR branch with main's primitive type conversion support
  • Main's implementation correctly converts numbers/booleans to strings in templates, preserving broader type support
  • Kept detailed examples explaining dot notation traversal behavior

test/unit/traefik/helpers.test.ts

  • Adopted main's comprehensive test suite (19 tests vs 8)
  • Includes edge cases: undefined parameters, different value types, key ordering preservation

Ancillary changes

  • Fixed redundant type annotation in config.ts (used as const instead of explicit type + value)
  • Normalized test suite naming convention
  • Removed implementation detail comment from test

Verification

  • 230 tests passing
  • No lint errors, type errors, or security vulnerabilities

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

stonegray and others added 12 commits January 12, 2026 03:49
* restart on config change

* add secure API

* add API key checking, timeouts, CORS, etc to API

* Add tests for API

* minor API updates, docs

* bugfix: can't traverse host directory

* ci: run on any PR

* eslint: fix any types

* bugfix: build types

* build: force npm build in lint phase

* eslint: add docs for overrides

* resolve merges

---------

Co-authored-by: Stone Gray <stone@stone.local>
* restart on config change

* add secure API

* add API key checking, timeouts, CORS, etc to API

* Add tests for API

* minor API updates, docs

* bugfix: can't traverse host directory

* ci: run on any PR

* eslint: fix any types

* bugfix: build types

* build: force npm build in lint phase

* eslint: add docs for overrides

* resolve merges

* Add runtime checks to templates to prevent various test failures

* Add test coverage for 3dc4bcd

* Update test/unit/traefik/template-validation.test.ts

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Stone Gray <stone@stone.local>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Traefik: Allow nested access to properties
* update readme

* remove old test stuff

---------

Co-authored-by: Stone Gray <stone@stone.local>
Co-authored-by: stonegray <7140974+stonegray@users.noreply.github.com>
…ix config types

Co-authored-by: stonegray <7140974+stonegray@users.noreply.github.com>
Co-authored-by: stonegray <7140974+stonegray@users.noreply.github.com>
Address review feedback: enhance type safety and test coverage
Copilot AI and others added 2 commits January 15, 2026 18:35
Co-authored-by: stonegray <7140974+stonegray@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix deep merge issue and enhance config robustness Merge main branch and resolve conflicts in templateParser and helpers tests Jan 15, 2026
Copilot AI requested a review from stonegray January 15, 2026 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants